Skip to content

GH-1460: Log the allocation manager inferred from the classpath instead of hardcoding log with netty - #1461

Open
raulcd wants to merge 1 commit into
apache:mainfrom
raulcd:GH-1460
Open

raulcd wants to merge 1 commit into
apache:mainfrom
raulcd:GH-1460

Conversation

@raulcd

@raulcd raulcd commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

What's Changed

Previous log was wrong and hardcoded:

12:40:22.176 [main] INFO  o.a.a.m.DefaultAllocationManagerOption - allocation manager type not specified, using netty as the default type
12:40:22.178 [main] INFO  o.apache.arrow.memory.CheckAllocator - Using DefaultAllocationManager at memory-unsafe/20.0.0-SNAPSHOT/arrow-memory-unsafe-20.0.0-SNAPSHOT.jar!/org/apache/arrow/memory/unsafe/DefaultAllocationManagerFactory.class

New log shows for memory-unsafe:

13:13:56.672 [main] INFO  o.apache.arrow.memory.CheckAllocator - Using DefaultAllocationManager at memory/memory-unsafe/target/classes/org/apache/arrow/memory/unsafe/DefaultAllocationManagerFactory.class
13:13:56.680 [main] INFO  o.a.a.m.DefaultAllocationManagerOption - allocation manager type not specified, inferred org.apache.arrow.memory.unsafe.DefaultAllocationManagerFactory from the classpath

For netty:

13:13:51.165 [main] INFO  o.apache.arrow.memory.CheckAllocator - Using DefaultAllocationManager at memory/memory-netty/target/classes/org/apache/arrow/memory/netty/DefaultAllocationManagerFactory.class
13:13:51.219 [main] INFO  o.a.a.m.DefaultAllocationManagerOption - allocation manager type not specified, inferred org.apache.arrow.memory.netty.DefaultAllocationManagerFactory from the classpath

Closes #1460.

@raulcd raulcd added the bug-fix PRs that fix a big. label Nov 12, 2026
@github-actions github-actions Bot added this to the 20.0.0 milestone Nov 12, 2026
@lidavidm

Copy link
Copy Markdown
Member

I'll let JB decide on merge since AFAIK he's still working on 20.0.0/declared a freeze

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix PRs that fix a big.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Message for default allocator using netty contradicts real allocator used (memory-unsafe)

2 participants